-
Notifications
You must be signed in to change notification settings - Fork 1.7k
feat(retry): add configurable delay and retry limits #4704
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
🦋 Changeset detectedLatest commit: 1ce2e8c The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
105c827 to
0c7fbe4
Compare
4c06668 to
0c2e8e2
Compare
|
Thanks @dannycreations! Due to the merge from upstream there are some merge conflicts. I will resolve those and review soon. |
ccde0ab to
26082b6
Compare
|
@kevinvandijk done pushing new code already tested all working |
- Implement retry logic in Task core to respect new configurations - Update settings UI with sliders for delay and retry count - Add localization for new settings across all supported languages - Add unit tests for auto-retry logic
26082b6 to
1ce2e8c
Compare
|
@marius-kilocode done implement every problem, im also remove all strategy to be "constant" by default since user can now change the retry delay, im also not touch any cli things and it should be fallback to default |
Implementation
Screenshots